Skip to content

feat(formatting): format generated diff corrections - #16209

Open
rgrinberg wants to merge 1 commit into
ocaml:mainfrom
rgrinberg:push-ywrznspwmqkx
Open

feat(formatting): format generated diff corrections#16209
rgrinberg wants to merge 1 commit into
ocaml:mainfrom
rgrinberg:push-ywrznspwmqkx

Conversation

@rgrinberg

@rgrinberg rgrinberg commented Aug 25, 2026

Copy link
Copy Markdown
Member

Starting with Dune 3.25, format generated corrections consumed by optional text
diffs before they are compared or promoted. Select Dune's built-in formatter
or OCamlFormat according to the source's formatting configuration, including
the locked OCamlFormat dev tool when available.

Apply this to user diff? actions and Cinaps without exposing generated
corrections as separate build targets. Construct formatted diffs directly during
action expansion so formatter dependencies, environments, and sandbox
requirements remain attached to the action. Older language versions retain the
previous behavior.

@rgrinberg
rgrinberg force-pushed the push-ywrznspwmqkx branch 8 times, most recently from 965a344 to 380f74d Compare August 28, 2026 15:47
Starting with Dune 3.25, format generated corrections consumed by optional
text diffs before comparing or promoting them. Select Dune's built-in
formatter or OCamlFormat according to the source's formatting configuration,
and use the locked OCamlFormat dev tool when available.

Construct formatted diffs directly during action expansion so formatter
dependencies, environments, and sandbox requirements remain attached to the
action. Apply the behavior to user diff? actions and Cinaps without exposing
generated corrections as separate build targets. Projects using older language
versions retain the previous behavior.

Signed-off-by: Rudi Grinberg <me@rgrinberg.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant